3392572 – git nasm fails to build lmacros tests

We are migrating the bug tracker to github Issues. This is now the preferred way to report NASM bugs.

Self-registration is disabled due to spam issue (mail gorcunov@gmail.com or hpa@zytor.com to create an account)

Bug 3392572 - git nasm fails to build lmacros tests

Summary: git nasm fails to build lmacros tests

Status:	CLOSED FIXED

Alias:	None

Product:	NASM
Classification:	Unclassified
Component:	Assembler (show other bugs)
Version:	2.15.xx
Hardware:	Other x86 Linux

Importance:	Medium blocker
Assignee:	nobody

URL:

Depends on:
Blocks:

Reported:	2019-06-06 09:53 PDT by E. C. Masloch
Modified:	2019-08-09 02:50 PDT (History)
CC List:	5 users (show)

Obtained from:	Built from git using configure
Generated by:	---
Bug category:
Observed for:	---
Regression:	---
Regression since:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description E. C. Masloch 2019-06-06 09:53:09 PDT

Trying to run my macro collection's tests with NASM built from the most recent git commit https://repo.or.cz/nasm.git/commitdiff/0995aa24aa7999a98f54c66ad989b47ce9a41e78 results in the following errors:

$ nasm -v
NASM version 2.12.01
$ nasm 001.asm -l 001.lst -f bin -o 001 -I ../
$ oldnasm -v
NASM version 2.15rc0 compiled on Dec 28 2018
$ oldnasm 001.asm -l 001.lst -f bin -o 001 -I ../
$ newnasm -v
NASM version 2.15rc0 compiled on Jun  6 2019
$ newnasm 001.asm -l 001.lst -f bin -o 001 -I ../
../lmacros1.mac:161: warning: macro `__REL__' exists, but not taking 1 parameters [-w+macro-params]
../lmacros1.mac:162: warning: macro `__REL__' exists, but not taking 1 parameters [-w+macro-params]
../lmacros1.mac:163: warning: macro `__REL__' exists, but not taking 1 parameters [-w+macro-params]
../lmacros1.mac:164: warning: macro `__REL__' exists, but not taking 1 parameters [-w+macro-params]
001.asm:32: fatal: (lret:30) Invalid type specified: auto
$ 

This is to be run from the tests subdirectory of https://bitbucket.org/ecm/lmacros/src/ecb478375acddc4f751788aeafe24bee6980d783/

Comment 1 E. C. Masloch 2019-06-06 10:34:50 PDT

This commit works fine: https://repo.or.cz/nasm.git/commitdiff/437e0ffa01505d173a8b9cfe2decf74f2e9795a5

Comment 2 H. Peter Anvin 2019-08-07 17:56:01 PDT

I believe this bug has been fixed. Please verify if it works correctly for you.

Thanks!

Comment 3 E. C. Masloch 2019-08-08 08:00:06 PDT

With nasm-2.14.03rc2-346-g80ba65e8 the tests seem to run fine.

Comment 4 E. C. Masloch 2019-08-08 08:03:44 PDT

Wait, I tested the wrong version. Now I redid it, the tests do seem to succeed, but these warnings do remain:

$ nasm 002.asm -I ../ -o 002.new
../lmacros1.mac:161: warning: macro `__REL__' exists, but not taking 1 parameters [-w+macro-params]
../lmacros1.mac:162: warning: macro `__REL__' exists, but not taking 1 parameters [-w+macro-params]
../lmacros1.mac:163: warning: macro `__REL__' exists, but not taking 1 parameters [-w+macro-params]
../lmacros1.mac:164: warning: macro `__REL__' exists, but not taking 1 parameters [-w+macro-params]

Comment 5 E. C. Masloch 2019-08-09 02:50:07 PDT

With https://repo.or.cz/nasm.git/commitdiff/1c21a53e4ed03371df5d9f16359545862bb4820e the warnings disappear, and all 16 tests produce the same results as with the 2.14.02 from Debian testing.